-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use a list of placements instead of one. (faster builder) #4471
base: 1.19.4
Are you sure you want to change the base?
Conversation
Interesting. Does this work on 2b2t? |
It should, but not tested. |
…faster. Use a list of placements instead of one. Which means we place blocks faster.
erm, what the sigma |
Is there something I need to do for this to have an effect? I just tried testing this and can't tell whether I was placeboing myself or it's actually faster. |
The best example I can find is placements that were already fast are just faster. |
For example, if I let Baritone place a ring of blocks, all placeable from the initial position, does this pr make it faster? I'm wondering because |
It's been over two months since I made those changes but I believe it does. Because it's running the right click function more often because it returns a list of blocks to place. |
Yeah, sorry for being a slow-ass reviewer. I had some higher priority things to spend my time on, so couldn't test this properly.
That's specifically the part I'm concerned about. The function it calls only sets a flag which says "RMB is pressed" and the packet will actually be sent later when |
You're the fastest maintainer I wouldn't worry about speed, its not like you record videos of you clipping your toe nails like leijurv either. But I don't think I have a way to test this, but I remember it was visibly faster. However its been so long I don't remember. Likely @mankool0 could explain why, he's the one that gave me the tip that this could be slow. |
What exactly is the problem with toe nail clipping videos??? 😂 |
Oh so now you respond, 😈 |
@mankool0, do you have anything to say on this? |
I'm not sure if this would work on 2b2t. I just know from testing on constantiam that in some cases you can place 2 blocks in a single tick if the rotation between them is minimal. I also haven't tested it with different blocks, but I imagine you'd have problems trying to place 2 different blocks at once. Since baritone is mostly 2b2t related I feel like this pr still needs changes to make sure the blocks it's trying to multi place are the same, otherwise it'll be placing the wrong blocks and will constantly be breaking and replacing. It should also be tested on 2b2t to see what the limit on rotations is and if this works there at all. I'm not currently able to test this pr so I'm not sure if other changes are needed. |
Is there anyone willing to test on 2b? |
This allows us to have multiple placement locations at once to speed up building.